xl: tell the user about non-existent domains
authorStefano Stabellini <sstabellini@xensource.com>
Mon, 9 Aug 2010 16:07:50 +0000 (17:07 +0100)
committerStefano Stabellini <sstabellini@xensource.com>
Mon, 9 Aug 2010 16:07:50 +0000 (17:07 +0100)
The error message when one wants to list a non-existent domain is at
best misleading (libxl_domain_info failed (code -5)).
This patch catches this specific error and tells the user that the
requested domain does not exist:
Error: Domain '42' does not exist.

Signed-off-by: Andre Przywara <andre.przywara@amd.com>
Signed-off-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
tools/libxl/xl_cmdimpl.c

index e42ad69cbe0701727209073f924e603ca2cc6d00..a2ccfe2c16a73d944b6db69c718fe9f0c41a07aa 100644 (file)
@@ -3003,6 +3003,11 @@ int main_list(int argc, char **argv)
     } else if (optind == argc-1) {
         find_domain(argv[optind]);
         rc = libxl_domain_info(&ctx, &info_buf, domid);
+        if (rc == ERROR_INVAL) {
+            fprintf(stderr, "Error: Domain \'%s\' does not exist.\n",
+                argv[optind]);
+            return -rc;
+        }
         if (rc) {
             fprintf(stderr, "libxl_domain_info failed (code %d).\n", rc);
             return -rc;